From: Colin Walters <walters@verbum.org>
Date: Fri, 4 Sep 2009 23:58:37 +0000 (-0400)
Subject: Bug 594184 - Fix up glow positioning logic
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~10962
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=fa5fb6b8a8a1fefb68728e983c9d98b89ebcc1f5;p=gnome-shell.git

Bug 594184 - Fix up glow positioning logic

We were allowing the glow allocation to go too far to the right;
clean up the logic.
---

diff --git a/js/ui/appIcon.js b/js/ui/appIcon.js
index 5e68839cd..46bce10bf 100644
--- a/js/ui/appIcon.js
+++ b/js/ui/appIcon.js
@@ -89,10 +89,10 @@ AppIcon.prototype = {
         let targetWidth = availWidth;
         let xPadding = 0;
         if (naturalWidth < availWidth) {
-            xPadding = (availWidth - naturalWidth) / 2;
+            xPadding = Math.floor((availWidth - naturalWidth) / 2);
         }
-        childBox.x1 = Math.floor(xPadding);
-        childBox.x2 = availWidth;
+        childBox.x1 = xPadding;
+        childBox.x2 = availWidth - xPadding;
         childBox.y1 = GLOW_PADDING_VERTICAL;
         childBox.y2 = availHeight - GLOW_PADDING_VERTICAL;
         this._name.allocate(childBox, flags);
@@ -101,7 +101,8 @@ AppIcon.prototype = {
 
         if (this._glowBox != null) {
             let glowPaddingHoriz = Math.max(0, xPadding - GLOW_PADDING_HORIZONTAL);
-            childBox.x1 = Math.floor(glowPaddingHoriz);
+            glowPaddingHoriz = Math.max(GLOW_PADDING_HORIZONTAL, glowPaddingHoriz);
+            childBox.x1 = glowPaddingHoriz;
             childBox.x2 = availWidth - glowPaddingHoriz;
             childBox.y1 = 0;
             childBox.y2 = availHeight;